Skip to content

Add runtime extra instruction files to pvmap generator prompt#1972

Merged
rohitkumarbhagat merged 2 commits intodatacommonsorg:masterfrom
rohitkumarbhagat:a12n-extra-instruction-files
Apr 27, 2026
Merged

Add runtime extra instruction files to pvmap generator prompt#1972
rohitkumarbhagat merged 2 commits intodatacommonsorg:masterfrom
rohitkumarbhagat:a12n-extra-instruction-files

Conversation

@rohitkumarbhagat
Copy link
Copy Markdown
Contributor

Summary

  • add --extra_instruction_files and --extra_instruction_max_bytes to pvmap_generator.py
  • validate extra instruction files before prompt generation
  • pass absolute extra-instruction file paths into the rendered prompt
  • update the prompt to read extra instruction files when present, with clear conflict preference
  • add a final execution reminder for extra instruction files

@rohitkumarbhagat rohitkumarbhagat enabled auto-merge (squash) April 20, 2026 18:47
@rohitkumarbhagat rohitkumarbhagat requested a review from ajaits April 20, 2026 18:47
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces the ability to provide extra instruction files to the Gemini agent during the PVMap generation process. It includes new configuration flags, validation logic for file size and encoding, and updates to the prompt template to incorporate these instructions. Feedback was provided regarding the enforcement of working directory boundaries for these extra files to ensure they are accessible when sandboxing is enabled.

Comment thread tools/agentic_import/pvmap_generator.py
Comment thread tools/agentic_import/pvmap_generator.py
@rohitkumarbhagat
Copy link
Copy Markdown
Contributor Author

Thanks Ajai for the feedback!!

@rohitkumarbhagat rohitkumarbhagat merged commit 0821d1f into datacommonsorg:master Apr 27, 2026
9 checks passed
@rohitkumarbhagat rohitkumarbhagat deleted the a12n-extra-instruction-files branch April 27, 2026 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants